Open standalone markdown files, telemetry on by default, fix CPU spin - #6
Merged
Merged
Conversation
configureScrollView rescheduled itself unconditionally whenever the probe had no enclosing scroll view, so a detached probe re-queued itself forever and pegged the app at 100% CPU, starving the event loop until even quit stopped working. Bail out when the probe has no window (reattachment reschedules via viewDidMoveToWindow) and cap lookup retries at 40 spaced 50ms apart. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Declare .md/.markdown document types so Finder offers Loci as a viewer, route incoming file URLs to a per-file viewer window that reads the file in place, and surface an Add to Library toolbar action for when the user does want a copy imported. Markdown now renders as formatted text (headings, lists, quotes, fences, wikilink titles) on the viewer canvas instead of raw monospace, for library items too. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Anonymous analytics now default on; an explicit choice under the current or legacy defaults key still wins, and the Settings privacy toggle keeps working as the opt-out. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The standalone markdown window drew the app-wide light titlebar over the viewer's dark canvas; give it a darkAqua appearance and matching background. The zoom and ellipsis menus use the borderless-button style, which greedily absorbs free width in wide windows and strands their dropdown chevrons at the trailing edge — fix them at their content size. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.md/.markdowndocument types (Viewer role, Alternate rank) so Finder's "Open With" offers Loci; file URLs open in a dedicated viewer window that reads the file in place — no library import unless the user clicks the new Add to Library toolbar action[[wikilinks]]shown by title) instead of raw monospace — applies to library markdown items tooScrollFeelProbeViewre-queued itself forever when detached from a scroll view, pegging the app at 100% CPU and starving the event loop; retries are now bounded and stop when the probe leaves the windowTesting
swift build -c releaseclean;swift test23/23 passingopen -a Loci standalone-test.mdopens a 780×660 viewer window alongside the main window, frontmatter/headings/lists/code render formatted, process idles at 0% CPU🤖 Generated with Claude Code